Always trigger layout instead of trying to skip it based on item changes #240

rubensousa · 2024-07-12T00:59:59Z

Changes:

Layout pass is not skipped if adapter changes are out of bounds because it might still affect the entire layout structure
Manual selection requests are cancelled when new layout passes are triggered
Always focus the selected view on each layout to prevent alignment issues
onFocusChanged now triggers focus on the next available frame, to prevent java.lang.IllegalStateException: Cannot call removeView(At) within removeView(At) #241

…yout

… remove

…multiple times for the same layout pass

…ling

rubensousa added 12 commits July 12, 2024 02:59

Always trigger layout instead of trying to skip it based on item changes

89a5516

Do not check for animation state after layout completes

06ef3f1

Mark compose view as not focusable when appropriate

cc945e4

Request a new layout if previous smooth scroller was active during la…

3f2940f

…yout

Schedule view focus in the next frame since it can be called during a…

e0af998

… remove

Only cancel explicit pivot selections and not new pivot searches

4772a42

Decrease delay

6a136e1

Wait for layout before continuing

4cfcb2b

Move scroll control logic to onLayoutChildren since it can be called …

a833543

…multiple times for the same layout pass

Improve ViewHolder focus detection in assertion

bd4729d

Add .kotlin to gitignore

4526a1d

Align all items to center since the first one is recycled after scrol…

145b087

…ling

rubensousa merged commit b7c140f into master Jul 23, 2024
6 checks passed

rubensousa deleted the predictive_animations branch July 23, 2024 13:35

Provide feedback